Skip to content

Fix IL trimming warnings and iOS CI build failure - #364

Merged
winnerspiros merged 1 commit into
masterfrom
copilot/check-build-errors-and-warnings-again
May 28, 2026
Merged

Fix IL trimming warnings and iOS CI build failure#364
winnerspiros merged 1 commit into
masterfrom
copilot/check-build-errors-and-warnings-again

Conversation

Copilot AI commented May 28, 2026

Copy link
Copy Markdown

CI reported 1 iOS build error and 30 IL trimming warnings (IL2026/IL2057/IL2072/IL2075) across osu.Game and osu.Game.Tournament, plus an informational Windows runner redirect notice.

Trimming warnings

Added [DynamicallyAccessedMembers] or [UnconditionalSuppressMessage] to the 10 affected call sites:

Warning Location Fix
IL2072 SerialisedDrawableInfo.Type [DynamicallyAccessedMembers(PublicParameterlessConstructor)] on property — annotation flows to all callers
IL2072 WizardOverlay.ShowNextStep(), ScreenWhiteBox() ctor, IMod.CreateInstance() [UnconditionalSuppressMessage]
IL2075 ModSpeedHotkeyHandler.ChangeSpeed() [UnconditionalSuppressMessage]
IL2057 SkinInfo.CreateInstance(), RealmRulesetStore.prepareDetachedRulesets(), RulesetInfo.CreateInstance() [UnconditionalSuppressMessage]
IL2026 TournamentGameBase.readBracket(), GetSerialisedLadder(), StableInfo ctor, SaveChanges() [UnconditionalSuppressMessage]

iOS build error

dotnet workload install ios (no version pin) fetched Microsoft.iOS.Sdk.net10.0_26.5 from NuGet, which requires Xcode 26.5 — the runner only has 16.4. Added --skip-manifest-update to use the SDK manifests bundled with the installed .NET 10 toolchain instead.

Copilot AI changed the title Fix IL trimming warnings and iOS build failure Fix IL trimming warnings and iOS CI build failure May 28, 2026
Copilot AI requested a review from winnerspiros May 28, 2026 18:37
@winnerspiros
winnerspiros marked this pull request as ready for review May 28, 2026 19:00
Copilot AI review requested due to automatic review settings May 28, 2026 19:00
@winnerspiros
winnerspiros merged commit fe10fdf into master May 28, 2026
19 of 23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants